Skip to content

fix SDL_config.h errors on older glibc #28

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jul 18, 2025

Conversation

OrangeTide
Copy link

glibc 2.37 and earlier does not support wcslcpy, wcslcat, strlcpy, and strlcat.

Without implementing a more sophisticated detection, it is simpler to enable SDL's fallback functions for these often missing routines.

example of affected systems:

Linux Mint 21.3 and earlier
Debian 12 (bookworm) and earlier
Ubuntu 22.10 (kinetic) and earlier
Fedora 37 and earlier
Zorin 17.3 and earlier

Also, updates README.md with some basic instructions.

glibc 2.37 and earlier does not support wcslcpy, wcslcat, strlcpy, and
strlcat.

Without implementing a more sophisticated detection, it is simpler to
enable SDL's fallback functions for these often missing routines.

example of affected systems:

Linux Mint 21.3 and earlier
Debian 12 (bookworm) and earlier
Ubuntu 22.10 (kinetic) and earlier
Fedora 37 and earlier
Zorin 17.3 and earlier

Also, updates README.md with some basic instructions.
Copy link

@jayschwa jayschwa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for bringing this up. Let me know if my proposed changes will work for you. I don't have an easy way to test on old glibc at the moment.

OrangeTide and others added 2 commits July 17, 2025 21:39
Co-authored-by: Jay Petacat <[email protected]>
Co-authored-by: Jay Petacat <[email protected]>
@OrangeTide
Copy link
Author

Thanks for bringing this up. Let me know if my proposed changes will work for you. I don't have an easy way to test on old glibc at the moment.

This works for me.

@jayschwa jayschwa merged commit c8d1ade into allyourcodebase:main Jul 18, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants